Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Fixed error in writing list<struct<..>> to parquet #1019

Merged
merged 1 commit into from
May 28, 2022

Conversation

jorgecarleitao
Copy link
Owner

Small but important fix that follows up from #1014 and #1007.

@jorgecarleitao jorgecarleitao added bug Something isn't working no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog labels May 28, 2022
@codecov
Copy link

codecov bot commented May 28, 2022

Codecov Report

Merging #1019 (35cfee1) into main (7cc874f) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1019      +/-   ##
==========================================
+ Coverage   71.68%   71.75%   +0.06%     
==========================================
  Files         359      359              
  Lines       19880    19938      +58     
==========================================
+ Hits        14251    14306      +55     
- Misses       5629     5632       +3     
Impacted Files Coverage Δ
src/io/parquet/write/nested/rep.rs 92.72% <100.00%> (ø)
src/compute/merge_sort/mod.rs 89.10% <0.00%> (+1.74%) ⬆️
src/compute/sort/lex_sort.rs 74.77% <0.00%> (+7.65%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cc874f...35cfee1. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 0654a26 into main May 28, 2022
@jorgecarleitao jorgecarleitao deleted the fix_write_parquet branch May 28, 2022 05:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant